Skip to content

chore: Bump ZooKeeper to 3.9.6 - #832

Merged
NickLarsenNZ merged 2 commits into
mainfrom
chore/zookeeper-26.11.0
Sep 22, 2026
Merged

NickLarsenNZ merged 2 commits into
mainfrom
chore/zookeeper-26.11.0

Conversation

@Techassi

Copy link
Copy Markdown
Member

razvan
razvan previously approved these changes Sep 16, 2026

@razvan razvan left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm. one more or less unrelated random comment

Comment on lines 20 to -21
- 3.9.5
- 3.9.4

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I know this comes sudden and we didn't discuss it, but I'm starting to question the usefulness of testing with multiple Zookeeper versions.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm honestly the wrong person to judge this. Feel free to bring this up in a daily or during the on-site.

In the meantime: What is the best way forward for now?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

testing is on the agenda for the next on-prem. I'll bring it up there

@Techassi Techassi changed the title test: Bump ZooKeeper to 3.9.6 in integration tests chore: Bump ZooKeeper to 3.9.6 Sep 21, 2026
@Techassi Techassi moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Sep 22, 2026
@Techassi
Techassi requested a review from razvan September 22, 2026 07:29
@NickLarsenNZ
NickLarsenNZ added this pull request to the merge queue Sep 22, 2026
Merged via the queue into main with commit 8c71179 Sep 22, 2026
13 checks passed
@NickLarsenNZ
NickLarsenNZ deleted the chore/zookeeper-26.11.0 branch September 22, 2026 09:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Development: In Review

Development

Successfully merging this pull request may close these issues.

3 participants